Hello! Thanks for the PR. You're right about moving the rule check up in the loop. But I prefer that you avoid changing lines only for cosmetic reasons, especially when it's very subjective, like removing blank lines that separate code blocks.

Revert changing lines changes.

@jmishra01 Thanks. Also, we're calling list() both times so that self.rule_builders can be permanent, which means create_callback() can be called more than once, and also that way it can be observed when debugging. It's not a huge difference, but there is just no reason not to do so. Sorry, I should have mentioned it in my previous message.

@erezsh Thanks. Understood the reason behind calling list(). I'm closing this PR.

@jmishra01 You can re-open with just the rule-check. I'll be happy to merge that in.